Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rollup of 14 pull requests #33658

Merged
merged 29 commits into from
May 15, 2016
Merged

Rollup of 14 pull requests #33658

merged 29 commits into from
May 15, 2016

Conversation

dns2utf8 and others added 29 commits May 10, 2016 11:06
- Adding name attribute to the sample code - Fix description sentences
For example, `Vec::len` is both a field and a method, and usually
encountering `vec.len` just means that the parens were forgotten.

Fixes: rust-lang#26472
It isn't "newly introduced" anymore.
typeck: if a private field exists, also check for a public method

For example, `Vec::len` is both a field and a method, and usually encountering `vec.len` just means that the parens were forgotten.

Fixes: rust-lang#26472

NOTE: I added the parameter `allow_private` to `method::exists` since I don't want to suggest inaccessible methods. For the second case, where only the method exists, I think it would make sense to set it to `false` as well, but I wanted to preserve compatibility for this case.
…tion, r=steveklabnik

Add error description for E0455

r? @GuillaumeGomez.

About this error there is no much thing to explain. The short description says enough to understand. Feel free to review.
Add error explanations for E0374, E0375, E0376 on issue rust-lang#32777
Simplify text

This way it should be clear: Any number of other threads have this guaranty not just one other thread.
Fix typo in std::sync::Once documentation
Cleanup formatting and wording for `std::env::temp_dir` docs.

None
…nts, r=steveklabnik

Add more details and examples in error codes

r? @steveklabnik
update "reason" for fnbox feature gate

It isn't "newly introduced" anymore.
…laumeGomez

Use the correct word in the explanation

r? @steveklabnik
doc: Fix comment in std::string::String example code
@rust-highfive
Copy link
Collaborator

r? @pnkfelix

(rust_highfive has picked a reviewer for you, use r? to override)

@Manishearth
Copy link
Member Author

@bors r+ p=10

@bors
Copy link
Contributor

bors commented May 15, 2016

📌 Commit 95ace6b has been approved by Manishearth

@bors
Copy link
Contributor

bors commented May 15, 2016

⌛ Testing commit 95ace6b with merge 9f58fb7...

bors added a commit that referenced this pull request May 15, 2016
@bors bors merged commit 95ace6b into rust-lang:master May 15, 2016
@Centril Centril added the rollup A PR which is a rollup label Oct 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rollup A PR which is a rollup
Projects
None yet
Development

Successfully merging this pull request may close these issues.